Skip to content

docs(desktop): rc.1 ad-hoc dry-run final report#644

Merged
jeevanpillay merged 2 commits intomainfrom
docs/desktop-rc1-dryrun-report
May 6, 2026
Merged

docs(desktop): rc.1 ad-hoc dry-run final report#644
jeevanpillay merged 2 commits intomainfrom
docs/desktop-rc1-dryrun-report

Conversation

@jeevanpillay
Copy link
Copy Markdown
Member

Summary

Final report for the rc.1 → rc.4 ad-hoc dry-run. Documents the seven bugs surfaced and fixed (PRs #638, #639, #640, #641, #642, #643), final pipeline state with rc.4 evidence, timing per cut, what remains for the first developer-id signed cut, and open items (Sentry quota, deferred CDP env var, unpinned root cause for renderer SDK no-op).

Premise of the dry-run held: every one of the seven bugs was invisible to local `pnpm package` and only surfaced after pushing real tags.

Test plan

  • Report renders cleanly in GitHub markdown
  • All PR/run/tag links resolve

🤖 Generated with Claude Code

Captures the seven bugs surfaced and fixed across rc.1 → rc.4:
slash in release id (#639), Forge tagPrefix (#639), Vite sourcemaps
(#640), debug-id injection timing (#641), renderer SDK no-op + IPC
bridge to main (#643), `releases new` before finalize (#642), and the
codesign correctness pre-fixes (#638). Documents pipeline timing,
what's left for the first developer-id cut, and open items.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown

vercel Bot commented May 6, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

3 Skipped Deployments
Project Deployment Actions Updated (UTC)
lightfast-app Ignored Ignored Preview May 6, 2026 8:17am
lightfast-platform Ignored Ignored Preview May 6, 2026 8:17am
lightfast-www Ignored Ignored Preview May 6, 2026 8:17am

Request Review

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 6, 2026

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (3)
  • thoughts/shared/2026-05-06-desktop-rc1-ad-hoc-dry-run-report.md is excluded by !thoughts/**
  • thoughts/shared/research/2026-04-23-codex-vs-lightfast-desktop-production-gap.md is excluded by !thoughts/**
  • thoughts/shared/research/2026-05-06-desktop-prod-readiness-status-verification.md is excluded by !thoughts/**

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 5d240fac-f571-460b-9a41-34328fb8ec28

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs/desktop-rc1-dryrun-report
✨ Simplify code
  • Create PR with simplified code
  • Commit simplified code in branch docs/desktop-rc1-dryrun-report

Comment @coderabbitai help to get the list of available commands and usage tips.

Codex-gap research: append "Status Update 2026-05-06 (post-dry-run)"
section closing G-1 (Sentry half), G-2, G-3, G-5, G-7 and adding
G-8..G-14 (the seven new gates surfaced and closed during the
rc.1 → rc.4 cuts). Points at the dry-run report.

Verification doc: re-stamp status as superseded-by-dry-run-2026-05-06,
add superseded_by frontmatter, insert a one-paragraph post-dry-run
callout right after the Summary so the supersession is visible from
the top.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@jeevanpillay jeevanpillay merged commit 6e6d5dc into main May 6, 2026
14 checks passed
@jeevanpillay jeevanpillay deleted the docs/desktop-rc1-dryrun-report branch May 6, 2026 08:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant